-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
배경화면 커스텀 #101
Merged
The head ref may contain hidden characters: "feature/\uBC30\uACBD\uD654\uBA74_\uCEE4\uC2A4\uD140_1\uCC28"
Merged
배경화면 커스텀 #101
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* theme 클래스에 이미지 관련 데이터 필드 추가 * 배경설정 화면으로 theme 데이터 가지고 들어가도록 수정 * 이미지 처리
tak8997
commented
Jan 24, 2025
presentation/src/main/java/com/nextroom/nextroom/presentation/ui/adminmain/ThemeListFragment.kt
Outdated
Show resolved
Hide resolved
data/src/main/java/com/nextroom/nextroom/data/datasource/SettingDataSource.kt
Show resolved
Hide resolved
presentation/src/main/java/com/nextroom/nextroom/presentation/ui/adminmain/BannerAdapter.kt
Show resolved
Hide resolved
presentation/src/main/java/com/nextroom/nextroom/presentation/ui/adminmain/ThemeListFragment.kt
Outdated
Show resolved
Hide resolved
* 배경화면 커스텀 정보 converter 추가
b122fa4
to
a4c5e48
Compare
...rc/main/java/com/nextroom/nextroom/presentation/ui/SubscriptionPaymentBottomSheetFragment.kt
Outdated
Show resolved
Hide resolved
...rc/main/java/com/nextroom/nextroom/presentation/ui/SubscriptionPaymentBottomSheetFragment.kt
Outdated
Show resolved
Hide resolved
data/src/main/java/com/nextroom/nextroom/data/network/response/ThemeDto.kt
Show resolved
Hide resolved
data/src/main/java/com/nextroom/nextroom/data/network/response/ThemeDto.kt
Show resolved
Hide resolved
* 다이얼로그를 호출한 화면에서 popbackstack을 연속으로 호출하려면 dismiss가 아닌 popbackstack을 호출해야함.
* 라이프싸이클 인자로 전달 * 리스너 해제
* 라이프싸이클에 따라 다시 그려져야 함
왜? 블러 처리에 시간이 많이 든다고 하여 추후 충분히 고려 후 기능을 넣기로 함 이번 버전에서는 제거
왜? 등록 이미지가 없으면 편집할 이미지도 없기 때문에 PC에서 이미지를 등록해달라는 토스트로 안내한다.
왜? 구독을 해서 여러개의 커스텀 배경을 사용하다가 만료 혹은 취소하는 경우를 대응해야함 이 경우 모든 테마의 활성화를 OFF 시키고 사용자가 직접 원하는 테마를 ON하도로 유도한다.
이전에 제거하기로 해서 뷰를 지웠는데 다른 로직 개선 작업 중 불필요하게 방해되어 제거함
로딩중에 뒷배경이 보이지 않아야 하는 화면을 위해서 백그라운드 타입을 설정할 수 있도록 개선함
왜? 뷰 하나 하나씩 로드되는 과정이 사용자에게는 사용성이 좋아보이지 않게 보이기 때문에 최초 로드시에는 검은 배경에 프로그레스바를 돌린다.
왜? 앱이 재시작 되었을 때 다른 정보는 복구되는데 커스텀 배경 정보는 백업/복구 하지 않는 문제가 있었음 GameState에 저장하여 복구하도록 함
왜? 기존에는 결제 완료 화면에 접근하는 경로가 하나 뿐이였음 하지만 앞으로 늘어날 것이므로 따로 분리 그리고 결제 완료 후 버튼을 누르면 홈으로 가게 되어있는데 이게 순환 참조를 일으킬 가능성이 있으므로 참조 관계를 끊고 popupTo를 이용함
왜? 일단 결제 진행 도중 로딩이 없어서 로딩을 넣었고 결제 완료/실패에 따라 이후에 해야하는 동작이 없어서 추가함
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
배경화면 커스텀 작업.
마지막 커밋은 force push로 바꿀예정